Skip to content

fix: Responsive calendar#9847

Open
snowystinger wants to merge 15 commits intomainfrom
responsive-calendar
Open

fix: Responsive calendar#9847
snowystinger wants to merge 15 commits intomainfrom
responsive-calendar

Conversation

@snowystinger
Copy link
Copy Markdown
Member

@snowystinger snowystinger commented Mar 26, 2026

Closes

Follows the pattern in the starter kits so that when the screen gets smaller, we can still fit the calendar without horizontal scrolling. To an extent. When things get really narrow, since we don't apply scaling down but rather only scaling up, on the prev/next buttons, they will become unaligned, this is expected.

Note, this only applies to a single visible month, if there are more visible months it should behave the same as before. Eventually, we'll determine the number of actual max visible months based on available space, not just the prop.

Important, if you're using Chrome dev tools to dynamic responsive resize, sometimes it can mess up, either refresh the page, or toggle the responsive mode off and on again, or do both things, see if it works then.

✅ Pull Request Checklist:

  • Included link to corresponding React Spectrum GitHub Issue.
  • Added/updated unit tests and storybook for this change (for new code or code which already has tests).
  • Filled out test instructions.
  • Updated documentation (if it already exists for this component).
  • Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

@github-actions github-actions bot added the S2 label Mar 26, 2026
@snowystinger snowystinger marked this pull request as ready for review March 28, 2026 03:02
# Conflicts:
#	packages/@react-spectrum/s2/src/RangeCalendar.tsx
@rspbot
Copy link
Copy Markdown

rspbot commented Mar 28, 2026

@rspbot
Copy link
Copy Markdown

rspbot commented Mar 28, 2026

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants